Skip to content

bug fix in volume constraint buffer value passed to Constraint plugin. - #1383

Merged
chrisjonesBSU merged 2 commits into
mosdef-hub:developfrom
chrisjonesBSU:quick-fix
Nov 8, 2026
Merged

chrisjonesBSU merged 2 commits into
mosdef-hub:developfrom
chrisjonesBSU:quick-fix

Conversation

@chrisjonesBSU

Copy link
Copy Markdown
Contributor

Use particle size for the volume constraint buffer value in the main loop instead of tolerance

PR Summary:

PR Checklist


  • Includes appropriate unit test(s)
  • Appropriate docstring(s) are added/updated
  • Code is (approximately) PEP8 compliant
  • Issue(s) raised/addressed?

@chrisjonesBSU chrisjonesBSU added the 2.0 Additions for mBuild 2.0 label Nov 8, 2026
@codecov

codecov Bot commented Nov 8, 2026 •

Copy link
Copy Markdown

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 82.71%. Comparing base (54f3075) to head (28559df).
⚠️ Report is 3 commits behind head on develop.

Additional details and impacted files
@@             Coverage Diff             @@
##           develop    #1383      +/-   ##
===========================================
+ Coverage    82.67%   82.71%   +0.03%     
===========================================
  Files           72       72              
  Lines         8747     8747              
===========================================
+ Hits          7232     7235       +3     
+ Misses        1515     1512       -3     

☔ View full report in Codecov by Harness.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

@chrisjonesBSU
chrisjonesBSU merged commit 23f168c into mosdef-hub:develop Nov 8, 2026
14 checks passed
@chrisjonesBSU
chrisjonesBSU deleted the quick-fix branch November 8, 2026 22:19
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

2.0 Additions for mBuild 2.0

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant